Skip to content

OBSDOCS-1860: Release Notes (RNs) for OpenShift Logging 6.3 Release #96021

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

theashiot
Copy link
Contributor

@theashiot theashiot commented Jul 11, 2025

Version(s): 6.3

Issue: OBSDOCS-1860

Link to docs preview: https://96021--ocpdocs-pr.netlify.app/openshift-logging/latest/about/logging-release-notes.html

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jul 11, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jul 11, 2025

@theashiot: This pull request references OBSDOCS-1860 which is a valid jira issue.

In response to this:

Version(s): 6.3

Issue: OBSDOCS-1860

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 11, 2025
Copy link

openshift-ci bot commented Jul 11, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 11, 2025
@theashiot
Copy link
Contributor Author

/test all

@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Jul 11, 2025

🤖 Wed Jul 16 12:14:08 - Prow CI generated the docs preview:

https://96021--ocpdocs-pr.netlify.app/openshift-logging/latest/about/logging-release-notes.html

@theashiot
Copy link
Contributor Author

/test all

@openshift-ci-robot
Copy link

openshift-ci-robot commented Jul 11, 2025

@theashiot: This pull request references OBSDOCS-1860 which is a valid jira issue.

In response to this:

Version(s): 6.3

Issue: OBSDOCS-1860

Link to docs preview: https://96021--ocpdocs-pr.netlify.app/openshift-logging/latest/about/logging-release-notes.html

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@theashiot theashiot force-pushed the OBSDOCS-1860 branch 2 times, most recently from a592b4b to 8ed6d77 Compare July 14, 2025 12:22
@theashiot theashiot marked this pull request as ready for review July 14, 2025 12:23
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 14, 2025
@theashiot
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Jul 14, 2025
@brendan-daly-red-hat
Copy link
Contributor

/label peer-review-in-progress

@openshift-ci openshift-ci bot added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Jul 15, 2025
* With this release, default values of Splunk metadata keys: index, indexed fields, source, and message payload are predefined for log forwarders. The values are based on the log type. As a user, you can override these values. (https://issues.redhat.com/browse/LOG-6859[LOG-6859])

=== Log storage
* With this release, you can specify the S3 secret in the `forcepathstyle` field. Use this field to configure Loki to use either path style or virtual host style S3 URL. By default, {aws-short} endpoints use the virtual host style URL, equivalent to setting the `forcepathstyle` attribute to `false`. (https://issues.redhat.com/browse/LOG-7024?[LOG-7024])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The first sentence sounds a bit weird. Maybe something like this?

Suggested change
* With this release, you can specify the S3 secret in the `forcepathstyle` field. Use this field to configure Loki to use either path style or virtual host style S3 URL. By default, {aws-short} endpoints use the virtual host style URL, equivalent to setting the `forcepathstyle` attribute to `false`. (https://issues.redhat.com/browse/LOG-7024?[LOG-7024])
* With this release, you can use the `forcepathstyle` field in the S3 secret. Use this field to configure Loki to use either path style or virtual host style for the S3 access. By default, only {aws-short} endpoints use the virtual host style URL, while others use path-style. (https://issues.redhat.com/browse/LOG-7024?[LOG-7024])

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Contributor

@brendan-daly-red-hat brendan-daly-red-hat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As Ashwin needs to add more bug fixes, I'm closing my review.

I'm onboarding as a peer reviewer, so please wait for @skopacz1 to approve my review before you proceed with my suggested changes.

[id="openshift-logging-release-notes-6-3-0-enhancements_{context}"]
== New features and enhancements

=== Log Collection
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use sentence-style capitalization

Suggested change
=== Log Collection
=== Log collection

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


=== Log Collection

* With this release, you can configure multiple {aws-first} outputs with distinct IAM roles in the `clusterLogForwarder` resource. (https://issues.redhat.com/browse/LOG-6790[LOG-6790])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion to define IAM, if applicable:

Suggested change
* With this release, you can configure multiple {aws-first} outputs with distinct IAM roles in the `clusterLogForwarder` resource. (https://issues.redhat.com/browse/LOG-6790[LOG-6790])
* With this release, you can configure multiple {aws-first} outputs with distinct identity and access management (IAM) roles in the `clusterLogForwarder` resource. (https://issues.redhat.com/browse/LOG-6790[LOG-6790])

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


* With this release, you can configure affinity rules to control collector scheduling. (https://issues.redhat.com/browse/LOG-6858[LOG-6858])

* With this release, default values of Splunk metadata keys: index, indexed fields, source, and message payload are predefined for log forwarders. The values are based on the log type. As a user, you can override these values. (https://issues.redhat.com/browse/LOG-6859[LOG-6859])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the colon:

Suggested change
* With this release, default values of Splunk metadata keys: index, indexed fields, source, and message payload are predefined for log forwarders. The values are based on the log type. As a user, you can override these values. (https://issues.redhat.com/browse/LOG-6859[LOG-6859])
* With this release, the default values of Splunk metadata keys (that is, index, indexed fields, source, and message payload) are predefined for log forwarders. The values are based on the log type. As a user, you can override these values. (https://issues.redhat.com/browse/LOG-6859[LOG-6859])

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

* With this release, default values of Splunk metadata keys: index, indexed fields, source, and message payload are predefined for log forwarders. The values are based on the log type. As a user, you can override these values. (https://issues.redhat.com/browse/LOG-6859[LOG-6859])

=== Log storage
* With this release, you can specify the S3 secret in the `forcepathstyle` field. Use this field to configure Loki to use either path style or virtual host style S3 URL. By default, {aws-short} endpoints use the virtual host style URL, equivalent to setting the `forcepathstyle` attribute to `false`. (https://issues.redhat.com/browse/LOG-7024?[LOG-7024])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion for clarity:

Suggested change
* With this release, you can specify the S3 secret in the `forcepathstyle` field. Use this field to configure Loki to use either path style or virtual host style S3 URL. By default, {aws-short} endpoints use the virtual host style URL, equivalent to setting the `forcepathstyle` attribute to `false`. (https://issues.redhat.com/browse/LOG-7024?[LOG-7024])
* With this release, you can specify the S3 secret in the `forcepathstyle` field. Use this field to configure the Loki Operator to use either the path style or the virtual host style S3 URL. By default, {aws-short} endpoints use the virtual host style URL, which is equivalent to setting the `forcepathstyle` attribute to `false`. (https://issues.redhat.com/browse/LOG-7024?[LOG-7024])

* Before this update, alerting rules created by the {loki-op} incorrectly used the `message` field to display the message related to the alert. With this update, the alerting rules correctly use the `description` field.
(https://issues.redhat.com/browse/LOG-6380[LOG-6380])

* Before this update, alerting rules were visible in unrelated namespaces due to lack of filtering based on the namespace in the Prometheus rules endpoint. As a consequence, user alerts were visible in unrelated namespaces. With this update, rule label filters have been added to the handler configuration, preventing alerts from being visible in unrelated namespaces. As a result, alert visibility is now restricted to the original namespace. (https://issues.redhat.com/browse/LOG-6148[LOG-6148])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestions to avoid duplication:

Suggested change
* Before this update, alerting rules were visible in unrelated namespaces due to lack of filtering based on the namespace in the Prometheus rules endpoint. As a consequence, user alerts were visible in unrelated namespaces. With this update, rule label filters have been added to the handler configuration, preventing alerts from being visible in unrelated namespaces. As a result, alert visibility is now restricted to the original namespace. (https://issues.redhat.com/browse/LOG-6148[LOG-6148])
* Before this update, because of a lack of filtering based on the namespace in the Prometheus rules endpoint, user alerts were visible in unrelated namespaces. With this update, rule label filters have been added to the handler configuration. As a result, alert visibility is now restricted to the original namespace. (https://issues.redhat.com/browse/LOG-6148[LOG-6148])

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice suggestion, thanks, updated!


* Before this update, alerting rules were visible in unrelated namespaces due to lack of filtering based on the namespace in the Prometheus rules endpoint. As a consequence, user alerts were visible in unrelated namespaces. With this update, rule label filters have been added to the handler configuration, preventing alerts from being visible in unrelated namespaces. As a result, alert visibility is now restricted to the original namespace. (https://issues.redhat.com/browse/LOG-6148[LOG-6148])

* Before this update, `ClusterLogForwarder` CR status updates failed due to an incorrect patching method. As a consequence, the {clo} failed to update objects, causing log data inconsistencies. With this release, `ClusterLogForwarder` CR status uses the `Patch()` method instead of the `Update()` method. As a result, the {clo} no longer fails to update the object, improving log forwarding stability. (https://issues.redhat.com/browse/LOG-6539[LOG-6539])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion for clarity:

Suggested change
* Before this update, `ClusterLogForwarder` CR status updates failed due to an incorrect patching method. As a consequence, the {clo} failed to update objects, causing log data inconsistencies. With this release, `ClusterLogForwarder` CR status uses the `Patch()` method instead of the `Update()` method. As a result, the {clo} no longer fails to update the object, improving log forwarding stability. (https://issues.redhat.com/browse/LOG-6539[LOG-6539])
* Before this update, `ClusterLogForwarder` CR status updates failed due to an incorrect patching method. As a consequence, the {clo} failed to update objects, which caused log data inconsistencies. With this release, `ClusterLogForwarder` CR status uses the `Patch()` method instead of the `Update()` method. As a result, the {clo} no longer fails to update the object, which improves log forwarding stability. (https://issues.redhat.com/browse/LOG-6539[LOG-6539])

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@theashiot theashiot force-pushed the OBSDOCS-1860 branch 2 times, most recently from 03342d1 to 78f33a4 Compare July 15, 2025 14:30
Copy link
Contributor

@skopacz1 skopacz1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is sort of a weird situation where Brendan only did a half review, but all of his comments look good to me. If you're still making changes to the PR, I think this would still need another peer review though

@brendan-daly-red-hat
Copy link
Contributor

/remove-label peer-review-needed
/remove-label peer-review-in-progress

@openshift-ci openshift-ci bot removed peer-review-needed Signifies that the peer review team needs to review this PR peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Jul 15, 2025
@theashiot
Copy link
Contributor Author

Thaks, @brendan-daly-red-hat and @skopacz1 for the review! Apologies for the trouble. I've now added the missing content. I agree that this needs to be reviewed again.

@theashiot
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Jul 15, 2025
@kabirbhartiRH
Copy link

@theashiot
Copy link
Contributor Author

Thanks, @brendan-daly-red-hat and @skopacz1 for the wonderful review! I've made all the suggested changes.

best,
ashwin

@briandooley
Copy link
Contributor

Should below issues be added under known issues? https://issues.redhat.com/browse/LOG-7311 https://issues.redhat.com/browse/LOG-7386 https://issues.redhat.com/browse/LOG-7375

These will be included in the next z-stream release.

@anpingli
Copy link

anpingli commented Jul 16, 2025

it is better to remove these bugs. these bugs had been released in Logging 6.2 z-stream.

LOG-6711 When forwarding logs with Otel data module, vector pod cannot forward ovn audit logs
LOG-6539 cluster-logging-operator: the object has been modified
LOG-6380 Alerting rules for log-storage contains summary and message annotation instead of summary and description annotation
LOG-6806 CLF errors out when dataModel is Otel and tuning config is empty
LOG-6987 Change Loki configuration to update memberlist ring when ingester becomes unhealthy
LOG-7028 [Logging 6.3] Parsing errors under Loki distributor for audit tenant when using Otel DataModel
LOG-7189 Syslog: merging data from 'message' to the root of log make log event inconsistent with ViaQ data model

@anpingli
Copy link

anpingli commented Jul 16, 2025

we should include this bug.
LOG-6148 Loki log based alerts for application tenant are not filtered by namespace on dev-console

@theashiot
Copy link
Contributor Author

we should include this bug. LOG-6148 Loki log based alerts for application tenant are not filtered by namespace on dev-console

This one's mentioned: https://github.com/openshift/openshift-docs/pull/96021/files#diff-615a1a88d52eafeb9b7cd91143bb5ea4e065d568bdd0fefc3d17829423b7328fR42

@theashiot
Copy link
Contributor Author

theashiot commented Jul 16, 2025

it is better to remove these bugs. these bugs had been released in Logging 6.2 z-stream.

LOG-6711 When forwarding logs with Otel data module, vector pod cannot forward ovn audit logs LOG-6539 cluster-logging-operator: the object has been modified LOG-6380 Alerting rules for log-storage contains summary and message annotation instead of summary and description annotation LOG-6806 CLF errors out when dataModel is Otel and tuning config is empty LOG-6987 Change Loki configuration to update memberlist ring when ingester becomes unhealthy LOG-7028 [Logging 6.3] Parsing errors under Loki distributor for audit tenant when using Otel DataModel LOG-7189 Syslog: merging data from 'message' to the root of log make log event inconsistent with ViaQ data model

I've commented these out. They are not visible in the output now: https://96021--ocpdocs-pr.netlify.app/openshift-logging/latest/about/logging-release-notes.html

Copy link

openshift-ci bot commented Jul 16, 2025

@theashiot: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@brendan-daly-red-hat
Copy link
Contributor

/remove-label peer-review-needed
/remove-label peer-review-in-progress
/label peer-review-done

@openshift-ci openshift-ci bot added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Jul 16, 2025
@theashiot
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Jul 16, 2025
@anpingli
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 16, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jul 16, 2025

@theashiot: This pull request references OBSDOCS-1860 which is a valid jira issue.

In response to this:

Version(s): 6.3

Issue: OBSDOCS-1860

Link to docs preview: https://96021--ocpdocs-pr.netlify.app/openshift-logging/latest/about/logging-release-notes.html

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@theashiot
Copy link
Contributor Author

Thanks, @anpingli!!

@gabriel-rh gabriel-rh added merge-review-in-progress Signifies that the merge review team is reviewing this PR and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Jul 16, 2025
Copy link
Contributor

@gabriel-rh gabriel-rh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/LGTM

@gabriel-rh gabriel-rh merged commit 9bb0269 into openshift:standalone-logging-docs-main Jul 16, 2025
2 checks passed
@gabriel-rh
Copy link
Contributor

/cherrypick standalone-logging-docs-6.3

@openshift-cherrypick-robot

@gabriel-rh: new pull request created: #96170

In response to this:

/cherrypick standalone-logging-docs-6.3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. merge-review-in-progress Signifies that the merge review team is reviewing this PR peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.